DM-33034: tools for building Sphinx docs from Pipelines #222
Conversation
TallJimbo
force-pushed
the
tickets/DM-33034
branch
from
2d13499
to
7d9c7b6
Compare
|
#347 has merged. I think that unblocks this ticket to allow it to be rebased on main rather than the temporary branch. |
|
Yes, but there will definitely be conflicts as well as other work to get this branch working again. Might be best to just close the PRs and then delete the temporary base branches. |
timj
force-pushed
the
tickets/DM-33034
branch
from
7d9c7b6
to
3727608
Compare
|
I dropped the commits from the base branch on the basis that you have already merged things (the expanded pipeline stuff was dropped?) and then the rebase was pretty straightforward once I also dropped all the ButlerURI renaming. |
|
The expanded pipeline stuff morphed dramatically into |
|
I've fixed up all the linting problems. @TallJimbo you may want to confirm it looks right for you before you delete your local branch. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #222 +/- ##
==========================================
- Coverage 81.23% 79.94% -1.30%
==========================================
Files 96 97 +1
Lines 10922 11099 +177
Branches 2085 2127 +42
==========================================
Hits 8873 8873
- Misses 1691 1868 +177
Partials 358 358 ☔ View full report in Codecov by Sentry. |
erinleighh
force-pushed
the
tickets/DM-33034
branch
from
9996bd5
to
cdc61bf
Compare
timj
force-pushed
the
tickets/DM-33034
branch
4 times, most recently
from
b4529da
to
c3d7525
Compare
Checklist
doc/changesThis PR is against a temporary branch, because merging this ticket (but not its review) is blocked by DM-33027. After that lands I'll rebase.
There are two largely mechanical commits here:
80c65ab just moves existing code from
ctrl_mpexec, with only trivial changes to import statements.b014171 is mostly a bulk-rename followup to DM-31723 - I did that here because using
ResourcePathin new code while leavingButlerURIin nearby existing code was getting increasingly awkward.The bulk of the changes are in 2d13499; everything else is just setup.